forked from langchain-ai/langchainjs
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update .nvmrc #1
Open
JeremyFabrikapp
wants to merge
1
commit into
main
Choose a base branch
from
broken-llm-node-18
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This solve an issue described here langchain-ai#1010
JeremyFabrikapp
pushed a commit
that referenced
this pull request
May 11, 2023
feat: add MyScale as vectorStore provider
JeremyFabrikapp
pushed a commit
that referenced
this pull request
May 11, 2023
* Merge post upstream sync (#1) * Added RedisMemory with tests and docs * fixed Redis test file name * TODOs for quick refactor * updated memory to accept a client * Added TODOS * ongoing testing * Updated tests * Updated tests for memory return option * finalized tests + updated docs * Adding docs * Readded init and cleaned up func * fixed test typing * redo yarn lock * remove yarn.lock * updated yarn lock and namespaced redis * fix merge conflict * updated BaseChatMemoryInput * Updated with lint fixes * Fixed docs to match memory instantiation * yarn format docs * Merging fixes to address ForkPR comments (langchain-ai#2) * updating for pr * Removed redis_memory in favor of chat memory * Fixed tests and updated docs * Bump docs * lint results * fixes from lint and format --------- Co-authored-by: Chris Toomey <[email protected]> * Update RedisChatMessageHistory class to initialize a client internally * Update Redis chat message history docs to reflect the fact that Redis is a peer dependency, allow direct passing of Redis client * Patch typing issues with passing node-redis client directly into a RedisChatMessageHistory instance --------- Co-authored-by: Chris Toomey <[email protected]> Co-authored-by: Jacob Lee <[email protected]>
JeremyFabrikapp
pushed a commit
that referenced
this pull request
Nov 5, 2023
JeremyFabrikapp
pushed a commit
that referenced
this pull request
Nov 5, 2023
…odelStart always in the same order as the prompts passed in (langchain-ai#1779) What was happening was that the runManager was only added to the managers array after the callback handlers finished executing, so eg if the callback handlers for the second run finished first the runManager for prompt langchain-ai#2 would actually be in position #1 of the returned array.
JeremyFabrikapp
pushed a commit
that referenced
this pull request
Nov 5, 2023
* Add Amazon Bedrock LLM endpoint integration (#1) A contribution from Prompt Security * Kept it as close to the Python implementation as possible * Followed the guidelines from https://github.com/hwchase17/langchainjs/blob/main/CONTRIBUTING.md and https://github.com/hwchase17/langchainjs/blob/main/.github/contributing/INTEGRATIONS.md * Supplied with unit test coverage * Added documentation * Fix bedrock documentation .mdx * Fix Bedrock mdx documentation to show embedded example * Fix Bedrock mdx documentation to show embedded example * Rename LLMInputOutputAdapter to BedrockLLMInputOutputAdapter as requested in PR review * Fixed @jacoblee93 suggestions from PR review * Added ability to specify credentials override via Bedrock class constructor. * Removed all dependency on `aws-sigv4-fetch` library and re-implemented the functionality using direct AWS api calls, directly within the `Bedrock._call()` function. * Rewrote unit-tests to mock fetch() instead of the (now missing `aws-sigv4-fetch` library). * Fixed documentation for Bedrock, hopefully better now. * Commit changes to yarn.lock after 'yarn install' in an attempt to fix build error 'YN0028: │ The lockfile would have been modified by this install, which is explicitly forbidden.' * Move hard deps to peer and optional deps, update docs --------- Co-authored-by: jacoblee93 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This solve an issue described here langchain-ai#1010